Skip to content

fix(react-card): change storybook asset url - #22713

Merged
theerebuss merged 1 commit into
microsoft:masterfrom
theerebuss:card-fix-story-assets
May 2, 2022
Merged

fix(react-card): change storybook asset url#22713
theerebuss merged 1 commit into
microsoft:masterfrom
theerebuss:card-fix-story-assets

Conversation

@theerebuss

Copy link
Copy Markdown
Contributor

Current Behavior

Story asset URL is still pointing to the old path

New Behavior

Story asset URL now pointing to /react-components/react-card

Related Issue(s)

#22692

@ghost

ghost commented Apr 29, 2022

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

@codesandbox-ci

Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit d6aa76b:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@fabricteam

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

🤖 This report was generated against 88fa8f279506c3cfbcc2facff95ecb5b420a217a

@fabricteam

Copy link
Copy Markdown
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 848 851 5000
Button mount 527 470 5000
FluentProvider mount 1805 1883 5000
FluentProviderWithTheme mount 272 301 10
FluentProviderWithTheme virtual-rerender 254 243 10
FluentProviderWithTheme virtual-rerender-with-unmount 289 299 10
MakeStyles mount 1417 1385 50000

@size-auditor

size-auditor Bot commented Apr 29, 2022

Copy link
Copy Markdown

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: 88fa8f279506c3cfbcc2facff95ecb5b420a217a (build)

@theerebuss theerebuss changed the title fix: change storybook asset url fix(react-card): change storybook asset url Apr 29, 2022
@theerebuss
theerebuss merged commit 14bafc5 into microsoft:master May 2, 2022
@theerebuss
theerebuss deleted the card-fix-story-assets branch May 2, 2022 13:47
marwan38 pushed a commit to marwan38/fluentui that referenced this pull request Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants